Skip to content

Conversation

@rubydong
Copy link
Collaborator

@rubydong rubydong commented Jun 11, 2025

Description

https://jira.mongodb.org/browse/CLOUDP-323670

News analytic event + an update to the toggle event

image
image
image
image

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added
  • If this change updates the UI, screenshots/videos are added and a design review is requested
  • I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement)

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@rubydong rubydong added the no release notes Fix or feature not for release notes label Jun 11, 2025
@github-actions github-actions bot added the feat label Jun 11, 2025
@rubydong rubydong added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Jun 11, 2025
collectionName: string;
indexNameTypeMap: Record<string, string | number>;
dataTestId?: string;
onCopy?: () => void;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do want this copy tracked for the index flow section? Right now we're only doing it for the query flow section.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh wait you're right let me check

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checked with analyst and she wanted both in and i realized i had the naming switched anyway so i fixed that. while i was there i added another event that was requested (index strategy documentation clicked)

@rubydong rubydong changed the title feat: Add analytics for input index copied + updated code equivalent toggle event CLOUDP-323670 feat: Add analytics for code copied, index strategy docs clicked, and updated code equivalent toggle event CLOUDP-323670 Jun 11, 2025
Copy link
Member

@Anemy Anemy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

Comment on lines 80 to 84
onCopy={() => {
if (onCopy) {
onCopy();
}
}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
onCopy={() => {
if (onCopy) {
onCopy();
}
}}
onCopy={onCopy}

@rubydong rubydong merged commit 6baba6a into main Jun 11, 2025
16 of 17 checks passed
@rubydong rubydong deleted the cloudp-323670 branch June 11, 2025 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants